Skip to content

feat(ci): include sql nudges in ci payload#95

Merged
veksen merged 1 commit intomainfrom
veksen/ci-include-nudges
Mar 25, 2026
Merged

feat(ci): include sql nudges in ci payload#95
veksen merged 1 commit intomainfrom
veksen/ci-include-nudges

Conversation

@veksen
Copy link
Member

@veksen veksen commented Mar 25, 2026

Summary

  • Add nudges: Nudge[] to CiQueryPayload interface and all QueryProcessResult variants
  • Capture nudges from analyzer.analyze() in processQuery() and pass them through to mapResultToQuery()
  • The analyzer already runs parseNudges() during AST traversal — this change plumbs the results into the CI API payload

Companion PR

Site/houston-v2: Query-Doctor/Site#2583 — adds nudges field to types, DB, and renders them in the CI query detail view.

These two PRs should be deployed together. The site PR uses optional().default([]) for backwards compat, so order doesn't matter.

Test plan

  • TypeScript check passes (tsc --noEmit)
  • Existing tests pass (integration tests require Docker)
  • Run analyzer against a test repo and verify the POST payload includes nudges array on each query

Closes Query-Doctor/Site#2583

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Query Doctor Analysis

View full run details

32 queries analyzed | 1 new query

2 pre-existing issues

Using assumed statistics (10000 rows/table). For better results, sync production stats.

@veksen veksen merged commit 9be464c into main Mar 25, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant